Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the admin teams UI to improve the “edit team” workflow, primarily by ensuring the edit action scrolls the user to the form and by normalizing judges data so the TeamForm can render/submit consistently.
Changes:
- Add an
handleEditTeamhelper to set the form data and scroll the view to the form when editing a team. - Normalize
data.judgesinTeamFormto consistently use full Judge objects (and validate/submit against the normalized data). - Adjust
JudgeCardrendering to avoid React key issues when a team may not have an_id.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| app/(pages)/admin/teams/page.tsx | Adds an edit handler that sets form state and scrolls the page/list to the top. |
| app/(pages)/admin/_components/Teams/TeamForm.tsx | Normalizes judges data for rendering/submission and validates using a derived formData. |
| app/(pages)/admin/_components/Judges/JudgeCard.tsx | Changes team list item keys to reduce key collisions when _id is missing. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…kDavis/hackdavis-hub into 465-fix-editing-team-details
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.